Skip to content

feat: support Responses upstream protocol - #80

Merged
orangeboyChen merged 3 commits into
mainfrom
codex/responses-upstream-protocol
Aug 20, 2026
Merged

feat: support Responses upstream protocol#80
orangeboyChen merged 3 commits into
mainfrom
codex/responses-upstream-protocol

Conversation

@orangeboyChen

Copy link
Copy Markdown
Owner

Summary

  • add per-credential Responses upstream protocol selection while preserving the legacy passthrough setting
  • translate Chat Completions and Anthropic message requests through the Responses API, including tools, reasoning, stop sequences, errors, and streaming usage
  • record streamed usage after terminal events and reader failures, with regression coverage for compatibility branches

Verification

  • bun run lint
  • bun run format:check
  • bun run typecheck
  • bun run test:coverage (219 tests, 93.57% statements)
  • bun run test:ci
  • bun run test:patch-branches --base c667adc (90.29%, 493/546)
  • bun run build --webpack

Environment note

The default Turbopack build cannot bind its internal local port in the managed Codex environment. The webpack production build completes successfully.

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.60146% with 35 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.54%. Comparing base (c667adc) to head (79311c7).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
- Coverage   94.67%   94.54%   -0.13%     
==========================================
  Files          20       20              
  Lines        3569     4090     +521     
  Branches      971     1184     +213     
==========================================
+ Hits         3379     3867     +488     
- Misses        190      223      +33     
Flag Coverage Δ
unittests 94.54% <93.60%> (-0.13%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@orangeboyChen
orangeboyChen marked this pull request as ready for review August 20, 2026 11:13
@orangeboyChen
orangeboyChen force-pushed the codex/responses-upstream-protocol branch from d19847c to 1edbc5d Compare August 20, 2026 11:14

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d19847c491

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread lib/server/proxy/codebuddy.ts
Comment thread lib/server/proxy/codebuddy.ts Outdated

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

View 2 additional findings in Devin Review.

Open in Devin Review

Comment thread lib/server/proxy/codebuddy.ts
@orangeboyChen
orangeboyChen merged commit c943e23 into main Aug 20, 2026
5 checks passed
@orangeboyChen
orangeboyChen deleted the codex/responses-upstream-protocol branch August 20, 2026 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant